-
Notifications
You must be signed in to change notification settings - Fork 40
[Nightly] Modify performance comparison #1646
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This reverts commit 197f60e.
Distributed test timeout is expected, UT failures are not related with this change |
Please work with @RUIJIEZHONG66166 to skip the temp ut failures. And for the e2e performance, I noticed that there are some performance regression as below. Does it cause by machine difference?
|
Should be caused by machine, need time to confirm. I will collect performance this weekend on all CI machine to check it |
Distributed test failed is expected |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, do we have any method to reset the best?
Launch a test manually and upload the best. |
Uh oh!
There was an error while loading. Please reload this page.